Skip to content

Conversation

@studioj
Copy link
Collaborator

@studioj studioj commented Sep 13, 2025

No description provided.

@studioj studioj requested review from a team and ssbarnea as code owners September 13, 2025 15:18
@studioj studioj added the ci label Sep 13, 2025
…name test methods for unittest compatibility
with:
ref: ${{ github.event.inputs.ref }}
# Use correct ref for PRs and pushes
ref: ${{ github.head_ref || github.ref_name || github.event.inputs.ref }}
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

github.head_ref is the pull request source branch, not the code that is requested for merge. Seems to me you are checking out the wrong version. Do you really need this or the inputs.ref at all? I'd expect it goes for the correct commit without the ref. If the input is needed, it's better available from inputs context as inputs.ref, no need to go for the github.event.inputs.

strategy:
matrix:
os: [ubuntu-latest]
# We only test a single version to prevent concurrent
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can add concurrency to prevent this workflow from running concurrently.

concurrency:
  group: jira-cloud-tests
  cancel-in-progress: false

push:
branches:
- main
- "*"
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't this the same as not specifying branches at all? Or do you plan to change it for main only?

on:
  push:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants